ufs-dev PR279#601
Conversation
scrasmussen
requested review from
dustinswales,
grantfirl,
hertneky and
mkavulich
as code owners
scrasmussen
force-pushed
the
ufs-dev-pr279
branch
from
3be6b5c to
9137d38
Compare
|
@scrasmussen Please pull in the latest main branch (#600 was merged). |
grantfirl
left a comment
grantfirl
left a comment
There was a problem hiding this comment.
syntax error in GFS_typedefs.F90
scrasmussen
force-pushed
the
ufs-dev-pr279
branch
2 times, most recently
from
ebb0642 to
86eb351
Compare
…an option to use liquid potential temperature in local mixing of GFSPBL
scrasmussen
force-pushed
the
ufs-dev-pr279
branch
from
86eb351 to
676e4b2
Compare
Mer
@grantfirl is this ok now? I couldn't find the syntax error you referred to and all the RTs are building now so I think it is ok? |
This has been completed 👍 |
hertneky
left a comment
hertneky
left a comment
There was a problem hiding this comment.
Looks good and builds now (phew, I thought I did something wrong last because I did not catch the letter swap, gj Grant!). All RTs on Ursa were successful
grantfirl
left a comment
grantfirl
left a comment
There was a problem hiding this comment.
Looks good now. Thanks!
|
Let's hold off on merging until Dom has had a chance to review the ccpp-physics PR. |
|
@scrasmussen NCAR/ccpp-physics#1148 was merged. Please update the ccpp-physics submodule pointer and .gitmodules in preparation for merge. |
scrasmussen
force-pushed
the
ufs-dev-pr279
branch
from
676e4b2 to
624d001
Compare
|
Noting that the |
3 participants
This PR catches the NCAR:main branch up with changes from the ufs-community:ufs/dev branch.
Associated ufs/dev PR:
Associated ufs-wm PR:
Associated fv3atm PR:
Associated NCAR PR: